Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

On search results, highlight searched word in item title - issue #11996 #16484

Merged
merged 1 commit into from Jun 13, 2017
Merged

Conversation

weeblr
Copy link
Contributor

@weeblr weeblr commented Jun 3, 2017

Pull Request for Issue #11996 .

Summary of Changes

Highlight a search result title, instead of only highlighting them in an article snippet.

Testing Instructions

See Issue #11996.

On stock Joomla, with sample data, search for the word "Joomla".

Expected result

At least one article has the word "Joomla" only in its title. This instance of "joomla" should be highlighted. Other instances in article snippets should still be highlighed.

Documentation Changes Required

#jab17

@carcam
Copy link

carcam commented Jun 3, 2017

I have tested this item ✅ successfully on 35752ef

Highlighting now works. Ordering needs a different issue and another pull requests as per the issue comments


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16484.

@coolcat-creations
Copy link
Contributor

I have tested this item ✅ successfully on 35752ef


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16484.

@ghost
Copy link

ghost commented Jun 4, 2017

RTC after two successful tests.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jun 4, 2017
@rdeutz rdeutz merged commit ffa0206 into joomla:staging Jun 13, 2017
@joomla-cms-bot joomla-cms-bot added PR-staging and removed RTC This Pull Request is Ready To Commit labels Jun 13, 2017
@rdeutz rdeutz added this to the Joomla 3.7.3 milestone Jun 13, 2017
@INOSIM
Copy link

INOSIM commented Jul 27, 2017

We found the following behaviour: When your search string was "joomla", and the original string in the title line was "JOOMLA", you get "joomla" as highlighted keyword in the title line of the search result. (Whereas, in the snippet from the article´s content below the title line in the search result, capitalization from the original content is maintained.) Hence, you do not get the original writing of the keyword in the title, but a modified writing using the search string which was entered.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16484.

@ghost
Copy link

ghost commented Jul 27, 2017

@INOSIM please have a Look at ##17241. Comments on closed issues didn't get much Notice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants